Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 20241125_nextjs_yarnberry_docker_김승태 #114

Merged
merged 2 commits into from
Dec 2, 2024

Conversation

dmdgpdi
Copy link
Contributor

@dmdgpdi dmdgpdi commented Nov 24, 2024

No description provided.

Copy link
Contributor

@tonyfromundefined tonyfromundefined left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

재밌게 읽었습니다!

제 생각에 nodeLinker를 빌드 타임에 바꾸는건 개발 환경과 운영 환경의 차이를 만들어서 복잡도를 높일거같은데요. 결국엔 코드에 쓰여진 nodeLinker 옵션을 node-modules로 바꾸고 개발 환경까지 통일하는게 더 나을거같아요 ㅎㅎ 포스팅과 관련된 내용은 아니여서 Approve 드립니다

@day1companyhaewooklee day1companyhaewooklee merged commit 2e8b647 into main Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants